src/features/phone-verification/model/use-phone-verification-status.ts (1)

147-196: ⚠️ Potential issue | 🔴 Critical

병합 충돌 마커가 해결되지 않았습니다.

useEffect 내 서버→스토어 동기화 로직과 의존성 배열에 충돌 마커가 남아 있어 CI가 실패합니다.

"Updated upstream" 브랜치가 resolvedServerIsVerified를 사용하고 usePhoneVerificationStore.getState()로 현재 상태를 가져오는 더 완성된 구현으로 보입니다. 반면 "Stashed changes"는 존재하지 않는 zustandIsVerified, zustandPhoneNumber 변수를 참조하고 있어 해당 부분을 사용하면 런타임 에러가 발생합니다.

🐛 충돌 해결 제안 (Updated upstream 기준)

  // 서버 → 스토어 동기화
  useEffect(() => {
    if (isProfileLoading || !userProfile || !memberId) return;

-<<<<<<< Updated upstream
    const current = usePhoneVerificationStore.getState();

    if (resolvedServerIsVerified) {
      // 서버가 인증됨 → 스토어 갱신 (memberId 포함)
      if (
        !current.isVerified ||
        current.memberId !== memberId ||
        current.phoneNumber !== serverPhoneNumber
      ) {
        store.setVerified(serverPhoneNumber ?? '', memberId);
-||||||| Stash base
-    if (serverPhoneNumber) {
-      // 서버에 전화번호가 있으면 Zustand도 동기화
-      if (!zustandIsVerified || zustandPhoneNumber !== serverPhoneNumber) {
-        setVerified(serverPhoneNumber);
-=======
-    if (serverPhoneNumber) {
-      // 서버에 전화번호가 있으면 Zustand도 동기화
-      if (!zustandIsVerified || zustandPhoneNumber !== serverPhoneNumber) {
-        setVerified(serverPhoneNumber, memberId ?? undefined);
->>>>>>> Stashed changes
      }
    } else {
      // 서버가 미인증 → 같은 유저 캐시 제거
      if (current.isVerified && current.memberId === memberId) {
        store.reset();
      }
    }
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, [
    isProfileLoading,
    userProfile,
    memberId,
    serverIsVerified,
-<<<<<<< Updated upstream
    serverPhoneNumber,
    resolvedServerIsVerified,
-||||||| Stash base
-    zustandIsVerified,
-    zustandPhoneNumber,
-    setVerified,
-    reset,
-=======
-    zustandIsVerified,
-    zustandPhoneNumber,
-    setVerified,
-    reset,
-    memberId,
->>>>>>> Stashed changes
  ]);

src/features/phone-verification/model/store.ts (1)

7-114: ⚠️ Potential issue | 🔴 Critical

파일 전체에 병합 충돌 마커가 해결되지 않았습니다.

이 파일에 여러 개의 병합 충돌 블록이 있어 CI 파이프라인이 실패합니다:

• Lines 7-21: PhoneVerificationState 인터페이스 정의
• Lines 35-69: 초기 상태 및 setVerified 구현
• Lines 74-85: reset 구현
• Lines 90-114: persist 설정

두 브랜치 모두 verifiedAt, memberId, hasHydrated를 추가하는 동일한 목표를 가지고 있습니다. 주요 차이점:

1. verifiedAt 타입: string | null vs Date | null - 서버 동기화를 위해 string (ISO 형식)이 더 적합합니다.
2. setVerified: spread 연산자 vs state 콜백 - state 콜백이 기존 memberId를 유지하므로 더 안전합니다.

🐛 충돌 해결 제안

 interface PhoneVerificationState {
   isVerified: boolean;
   phoneNumber: string | null;
-<<<<<<< Updated upstream
   verifiedAt: string | null;
   memberId: number | null;
   hasHydrated: boolean;
   setVerified: (phoneNumber: string, memberId?: number) => void;
-||||||| Stash base
-  verifiedAt: Date | null;
-  setVerified: (phoneNumber: string) => void;
-=======
-  verifiedAt: string | null;
-  memberId: number | null;
-  hasHydrated: boolean;
-  setVerified: (phoneNumber: string, memberId?: number) => void;
-  setHasHydrated: (hasHydrated: boolean) => void;
->>>>>>> Stashed changes
   reset: () => void;
   setHasHydrated: (hasHydrated: boolean) => void;
 }

 export const usePhoneVerificationStore = create<PhoneVerificationState>()(
   persist(
     (set) => ({
       isVerified: false,
-<<<<<<< Updated upstream
-      phoneNumber: null as string | null,
-      verifiedAt: null as string | null,
-      memberId: null as number | null,
-      hasHydrated: false,
-      setVerified: (phoneNumber, memberId) =>
-        set({
-||||||| Stash base
-      phoneNumber: '',
-      verifiedAt: new Date(),
-      setVerified: (phoneNumber) =>
-        set({
-=======
       phoneNumber: null,
       verifiedAt: null,
       memberId: null,
       hasHydrated: false,
       setVerified: (phoneNumber, memberId) =>
         set((state) => ({
->>>>>>> Stashed changes
           isVerified: true,
           phoneNumber,
-<<<<<<< Updated upstream
-          verifiedAt: new Date().toISOString(),
-          ...(memberId !== undefined ? { memberId } : {}),
-||||||| Stash base
-          verifiedAt: new Date(),
-=======
           verifiedAt: new Date().toISOString(),
           memberId: memberId ?? state.memberId,
         })),
-      setHasHydrated: (hasHydrated) =>
-        set({
-          hasHydrated,
->>>>>>> Stashed changes
-        }),
       reset: () =>
         set({
           isVerified: false,
-<<<<<<< Updated upstream
-          phoneNumber: null as string | null,
-          verifiedAt: null as string | null,
-          memberId: null as number | null,
-||||||| Stash base
-          phoneNumber: null,
-          verifiedAt: null,
-=======
           phoneNumber: null,
           verifiedAt: null,
           memberId: null,
->>>>>>> Stashed changes
         }),
       setHasHydrated: (hasHydrated) => set({ hasHydrated }),
     }),
     {
-<<<<<<< Updated upstream
-      name: 'phone-verification-storage',
-      partialize: (state) => ({
-||||||| Stash base
-      name: 'phone-verification-storage', // 로컬 스토리지에 저장
-=======
       name: 'phone-verification-storage',
       partialize: (state): PersistedPhoneVerificationState => ({
->>>>>>> Stashed changes
         isVerified: state.isVerified,
         phoneNumber: state.phoneNumber,
         verifiedAt: state.verifiedAt,
         memberId: state.memberId,
       }),
       onRehydrateStorage: () => (state) => {
         state?.setHasHydrated(true);
       },
     },
   ),
 );

추가 참고사항:

• null as string | null 형태의 타입 단언은 불필요합니다. TypeScript가 null을 자동으로 추론합니다.
• setVerified에서 state 콜백을 사용하는 "Stashed changes" 방식이 기존 memberId를 보존하므로 더 안전합니다.